Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add: Unscheduled with btc_delay_createUnit #799

Merged
merged 16 commits into from
Mar 31, 2020
Merged

Conversation

Vdauphin
Copy link
Owner

@Vdauphin Vdauphin commented Dec 26, 2019

  • Add: Unscheduled with btc_delay_createUnit (@Vdauphin).

When merged this pull request will:

  • title
  • This is a other (fourth...) try to reduce FPS drop when many units are created on city activation.
  • The basic idea is to create a unit when the previous one has been created 0.3s before.
  • Need AI: Don't halt planned movement for empty group CBATeam/CBA_A3#1274 (CBA 3.13.1 or later)
  • check waypoint generation
  • check if IA go to waypoint as intended
  • look at static clean up
  • check if group with all units dead are not saved
  • null group

Final test:

  • local
  • server

Vdauphin and others added 10 commits January 2, 2020 13:25
- fix city are clear too fast
- fix group empty never clean up
- fix empty group addWp
- Increase delay to 1s
-  unitcreate is delayed slightly lower (0.01s) to be sure those are created before others waitAndExecute
- Remove clearwaypoints for new unit in spawn_group
- delay set groupOwner

Co-Authored-By: pterolatypus <pterolatypus@users.noreply.github.com>
@Vdauphin Vdauphin added this to the 1.21 milestone Jan 18, 2020
@Vdauphin Vdauphin changed the title Add: Scheduled with btc_delay_createUnit Add: Unscheduled with btc_delay_createUnit Jan 20, 2020
@Vdauphin Vdauphin merged commit 6ad7168 into master_daily Mar 31, 2020
@delete-merged-branch delete-merged-branch bot deleted the Add-Scheduled_v2 branch March 31, 2020 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant